-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
i18n(es-ES) Spanish Spain translation #4495
Conversation
can you re-trigger the workflow? - seems to be stuck for a translation PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Joe,
The #4531 introduces a global list for available languages - I added two suggestions to align with this PR.
packages/node_modules/@node-red/editor-client/locales/es-ES/editor.json
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/editor-client/locales/en-US/editor.json
Outdated
Show resolved
Hide resolved
Thank you @GogoVega I made the suggested changes and synced the translations with latest changes on master branch |
great!! Thanks :-) |
Proposed changes
Spanish Spain translation
Checklist
npm run test
to verify the unit tests pass